at we do 


SOFTWARE к=, 


CRAFTSMANSHIP STOP 
CODE ROT 


LANGLEY AND WRITE BROTHERS 
TEAM S MINDSET MINDSET 


HAVE TO DO IT WANT TO DO IT 


IN 1982, A THEORY WAS 
INTRODUCED BY 


SOCIAL SCIENTISTS 
JAMES О. WILSON 


LITTER GRAFFITI 


WINDOW 


AR. 


STRUCTURE 


THE ROT FASTER 


NEGLECTION 


THAN ANY OTHER FACTOR. 


HOW IS THIS RELATED 
WITH THE SOFTWARE? 


Are the next few questions 
sounds familiar? 


CHANGES ARE HARD IO DO? 


SIMPLE CHANGE IMPACTS 
NUMEROUS MODULES? 


IMPLEMENTING SIMPLE 
CHANGE TAKES FOREVER? 


CHANGE IN ONE PLACE 
HARMS SOMEWHERE 
COMPLETELY UNEXPECTED 
AREA? 


FIXING A BUG CAUSES <N> 
MORE? 


MODULES ARE 
NOT REUSABLE 
BECAUSE OF THEIR 
DEPENDENCIES 7 


IT'S EASIER 
TO DO HACKS 
THAN BY THE BOOK? 


public static String testableH 
PageData pageData, 
boolean includeSuiteSetup 
) throws Exception { 
WikiPage wikiPage = pageDat 
StringBuffer buffer = new S 
if (pageData.hasAttribute (" 
if (includeSuiteSetup) { 
WikiPage suiteSetup = 
PageCrawlerImpl.getIn 
SuiteResponde 
); 
if (suiteSetup != null) 
WikiPagePath pagePath 
suiteSetup.getPageC 
String pagePathName = 


public List<ir 
List<int[]> 
for (int[] > 
if (х[0] = 
listl.ac 
return list] 


cubated GG: сылап Š Seng 
ee een 
Weg teg 
inia i ea be ee 


protected Loader loader = null; 


CODE ROT 
HAS BEGUN? 


SOFTWARE 
EROSION 


SOFTWARE 
DECAY 


SOFTWARE 
ENTROPY 


SOFTWARE 
ROT 


TECHNICAL 
DEBT 


WE KNOW 
ІІ BY MANY 
NAMES 


HOW CAN WE STOP IT? 


KEEP THE 
CODE CLEAN 
АТ ALL TIMES 


WHAT'S THE CLEAN CODE? 


.. DOES ONE THING WELL 


-- Bjarne Stroustrup 
Inventor o f C++ 


. READS LIKE WELL-WRITTEN PROSE 


-- Grady Booch 


.ІТ WAS WRITTEN BY SOMEONE 
WHO CARED 


-- Michael Feathers 


Author of Working effectively with Legacy Code 


.. FEW PROGRAMMERS KNOW 
HOW TO WRITE CODE THAT 
HUMAN UNDERSTAND 


-- Martin Fowler 
Author of Refactoring 


„IS WHEN EACH METHOD YOU READ 
TURNS OUT TO BE PRETTY MUCH 
WHAT YOU EXPECTED 


-- Ward Cunningham 


Inventor of Wiki 
Coinventor o f Extreme Programming 


d 


1) 


OUR GOAL IS 
NOT TO CODE 
BUT TO 
COMMUNICATE 


WHILE MAKING 
CHANGES 


WHILE ADDING 


TO EXISTING CODE UR Sewanee 


WHILE MAKING 
CHANGES 


TO EXISTING CODE 


ANY IDEA WHEN 
EXISTING CODE 
BECOMES LEGACY? 


THE MOMENT WE 
STOP REFACTORING 
OR STOP CARING 


MAINTENANCE START THE MOMENT 
WE WRITE [HE CODE 


WHAT HAPPENS WHEN NOT CARED 
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“DON'T PANIC 
| KNOW LITTLE REFACTORING 
LET’S DO IT!!” 


CREATE PROCEDURE dbo.prv BundleFilter set old 
EvalueDate DATETIME , 
@asOfDate DATETIME , 
@idBundle INT = NULL , 
@localizationsG INT = NULL و‎ 
@idFOCode INT = NULL , 
@isMigrated BIT = NULL , 
@idNCRStatus INT = NULL , 
@idCollateralStructure INT = NULL , 
@isActive INT = 1, 
@idProfile INT = NULL 


DECLARE ätolodesTable TABLE 
( 
idFrontOfficeCode INT 


INSERT INTO @foCodesTable 
SELECT * 
FROM fn_GetDistinctFOCodes (@idProfile) 


DECLARE @CombinedFacility TABLE 
( 
idBundle INT NULL INDEX IX idbundle, 
idBBMultiFacilityHeader INT NULL و‎ 
isActive BIT 


CREATE PUMCTION des. tu Dusdle create 
1 


-. the futimmary variable table Lu central Еп the function and Світе filled step by xtsp 
RETURNS йзшллшгү tabla | 
iBBundLs Let sull IRC ide idlundr 
ciudad int mall 
„lft int mall 
erat int nell 
cio res int null 
eiddututancsingWablo Let null 
р Ши гъ про int mall 
ridelo int null 
valute сіліюбіли cull 


eidttentype int mall 
„type varchear[ zdj null 
acta) mall 


= werchar (255) sull 
¿adios chard) mall 
¿dior СОЇ chart 1] mall 
cil 180 int null 

„тв varchar[ 155) mall 
„idTrmmtltrieslnda Let null 
ePramtittLerCode marchar 255] mull 
гілміштііші hit mall 

vir Bätetes ігі sull 


- Global Summary Infomation 
гіїніфляюиті tlost mall 
„Ала Алое ГОН) ticat null 
‚collateralfsount fLzat mull 
-Егиялс атга алаи tlnat nell 
„тш а ртт thot ruil 

- UMD Ey АГ : valo oistszirmmréo bm reeves) „сатен тата та легата лит? float na 
DutrtasdingCurrertAucunk ticat null 
,.PutrtasdizsgThsariralAscunt fleet mull -- ralati* вых B 
насі Іі byw 1 ва Amunt fleet rull -- Limit - 
eml titacilitpřvai lable tlnat nell «= 1 
= СО атха ЗЕ оплат thet muld -- [colle 
مسد‎ і تنس ناتك !م لها‎ AA nall -- 5 
efeciLitwiwellableCurrmtiocent Tlzat mull -- | E 
«multitael liiteid ШЕ ште Алии Flank nall -- Е 
со Рея ЗЕ ии стае Аи ticat mul] -- [col lat 
!م 1 مسمس‎ crtiruamsirgmUkemarrtimarurit Ғісші sell -- minita 


- 56 Summary Intermatior 

esil За Евлия Тісі mull 
„all За Ейлат Е СТО Loa muld 
„ ЗОСІ авага Шазима Float nall 
¿roo Ішов АлосатЕ са? rull 
тас urrerttollsterslässrt floss nell -- just ter shared BE current collat 
„skKCurrertzruuuCallateralinzunt Тісші гый] -- dut dor shared ШІ correct collet 
гтіншігінінгеіге Ascer float то 

-- UMW ty AT : valo ситкЕвгеЧпд( то be reeves) „sLostutancingintradanieszint Тісегі 
„stautatanciinęcurrentizaunt *Lzwt muld 
„sKlututansinąThearicrulAseust Float mall 

„ЗОГ вас 13 Ерйгла і ЕЛ а OA lost nall -- (Limit 
„Оли | З 3Г сі ЦЕ лъв! lablefüöfmsunt that ги! -- (1 
н ЗОСІ ав Раі З снаа ге Plo null -- (call 
рані lable orir mbaint Тісші mul] -- 
„ЗОГ исі litphrnilableluorrentämoert Flank sull -- (limi В 
Í,ümultirarilityMwailanLsCurrmrt/mzunt ticat mull -- - gut wtanding ji 
.ZLücnllakPzaitinncurremrtAscemt *laat sull -- (osdlat - оивнбюгкіі ві 

võtma lak Лебагіт ва бесы гие лсантЕ tiat null -- mini2acilityñnasilablatagurrt, arvalla 


uant, weed lable? 


„uiridCH Жісей ruli 
eänperteetlyõecured iait Тісей muld 


¿pertectiyiscurediacont float oull 
„ispartact bySecuredivelleble ticat null 
= ШН ГАЗАЛИИ flaak mull 


„perńazt Іубесигакійлюном СО Flos null 
„isparfart дъба сива абе СОС: Tlzat rull 
„Епвесъге ливлг flusk mall 


Í,supsertertlySerurscwcunt ticat null 
مايل‎ гас Грасси і lable *laat neli 
„Олга със Е ticat null 


тірісі асына ЕГІС tiat null 
візі лар гас: ly Secor i lable COO flask sull 
„тшге ساح‎ валс ЕС thot rull 


Bargint;allAscunt fLzat mull 


,hamHarrintallThrsaheld int roll 
ethreshcldtype char[1] sall 
‚threhzldvalue ticat null 
„idłaręincallńlert=czpe int mull 
„iudsCRAJerticape irt mull 


wg ët Lo Тісі mull 

fart СОМ ze fluat mall 
‚Kosalerreriflatie ticat null 
rmrtCurremtlkrtin flask null 


„idczllaterulitructure irt mol], 

iibutetamdärpätimsunt int mall, 

За lmitSDNmcamk ist soll, 

Shar ingSG float mall, 

ingSEPIU Float mall, 
imbicerrert flask null, 

shared ілі cull, 


id Snapahet irt mul], 

лт. ТЕ ілі sull, 

Cooke ticat null, 
Serndicatedicoke ticat null, 
Buta kandi ninal lost null, 
Enuzsdezrtipncpoks tlowt null, 
commited ілі sull, 
нгарияһсігінік datwtine null, 


терігүПшіш Satstbus, 
maturityDats datwtine, 
зе сіли нів САТ СТ ВАЄ 


-- faced Бү Anant for itrack 1284178 
,initialoutatasdirgsaAmcGent Float suli 
,iniLbkialOutatasdirgGlohalAscumnt tlzat rull 

.RrcaxEnelit5oLnllateraliatin FLOAT HULL 

„гхи а rebel Co) lateralRatio FLOAT MALL 
гео а ге oo eters Шолом: FLOAT ML 
„гаиа гъ 153 Со 1 аба Ленс Е FLOAT HULL 

F 

au 

begin 

return 


102 
COLUMNS 


100+ 
DEPENDENCIES 


io_ExportGlobalCollateral Facilities 
+ EI nr. ValuationOutstanding, get 
EE] nr. ValuationWaming get 


зир Data Extraction. At Facilitylevel 

Е] sup_Data_Extraction_DafPrimaryDetails 

Е] sup_Data_Extraction_DafPurchaseDetails 

E] sup_Data_Extraction_DafSaleDetails 

EE ts_AnalyticalData_RemoveEntriesForTransaction 


WAS STILL 
SCRATCHING 
THE SURFACE 


FOLLOWING 
THE EXISTING 


UNFAMILIAR 


INEXPERIENCED 
DEVELOPERS 


PATTERN зані 
"E —Q MAKING 
HE es a SMALLEST 
POSSIBLE CHANGE 
DON'T KNOW HARD 
KAUM UT WHEN ІТ To 


REGULATIONS 


HAPPENED 


UNDERSTAND 


WHY 

THIS SIORED 
PROCEDURE 
BECAME LIKE THIS? 


ATTEMPT I 


SMALL CHANGE 8 IT FAILED 


CODE IS RIGID 


ATTEMPT 20+ 


Made changes, it worked!!! 
Checked In. 


WE RELEASED © 


After 2 Days in production 
It broke the reports 
from completely different part of 
application 


CODE IS FRAGILE 


100+ 


DEPENDENCIES 


Dependencies 


= prv_BundleFiter_set 


io_ExportGlobalCollateralFacilities 
nr_ValuationOutstanding_get 
nr_ValuationWaming_get 

nr. ValuationWamingGreenLight, get 
prv_Multifacility_get 
rs_AnalyticalDataBalance_get 
rs_BundleLabel_get 

rs_ClientExport 
rs_CollateralDetail_get 

rs_Collateral Summary. get 

rs CombinedFacility Dashboard get 
rs Cooke get 

rs CounterpartExposure Customerlmpact, get 
rs CustomerDashboard get 

rs DocumentDetail get 


| rs_DocumentGreen Light_get 


rs Facility getAll 


-E в FacilityParameters 
-图 rs_GetCurrencyRate 
由 -图 rs_GetSub Limits 
EI вв. Global. getAllByExpiryDate 
-E в GlobalDashboard get 
-E rs_GlobalMTMLossAndGain 


rs_GlobalSummary_get 
rs_LiquidityReport_get 
rs_Monitoring_get 
rs_MultiExposure_get 
rs_NCR_DashBoard 
rs_OutstandingSummary_get 

rs OutstandingSummaryEXCEL, get 
rs. PortfolioReviewReport Get 

rs SalesContract get 

rs SpotApproval get 

rs_ Tickler_getActiveByldBundle 

rs TransactionDashboard get 

rs ValoDetail getRate 

rs WarehouseDetail get 

rs WarehouseDetailsAndExposure get 
rs_WamingGreenLight_get 
sup_Data_extraction_at_client_level 
sup_Data_Extraction_At_Facilitylevel 
sup_Data_Extraction_DafPrimaryDetails 
sup_Data_Extraction_DafPurchase Details 


-图 sup_Data_Extraction_DafSaleDetails 


ts AnalyticalData RemoveEntriesForTransaction 


When looked deeper into 
the code, dependencies 
Nave grown because this 
module 
couldn't be reused 


partly 


CODE IS IMMOBILE 


BHARAT 


Module was full of 
technical names than 
business/domain names 


CODE IS NOT READABLE 


PERFECT 
INGREDIENTS 
FOR? 


el rm 


ШИ: 


44 
37 


Ще 


An 


TEST AUTOMATION 


UNIT FUNCTIONAL 
TESTING TESTING 


DATABASE 


ZISONPATCH DIFF Y NONREG PACK 


Finds potential bugs In your system 
comparing the output of 
new code and your old code. 


N 


(л 


STORED PROCEDURE OUTPUT 


COMPARER 


STAND ALONE 


ud SPComparer 


Gbundleld = 137, @valueDate = 2019-03-22 
@bundleld = 143, @valueDate = 2019-03-22" 
©bundleld = 177, @valueDate = 2019-03-22 


Result Data is not same for Table and for Column idTransactionVersion 


BlockedCashAmow total Amount 


53416125 | 53416125 


Failed SP with Parameters 
dbo ts_FaciityBlockedCash_EconomicMargin @bundield = 101, @valueDate = 2019-03-22 


Results 
How many times to run? Parallel Users 


A | 


| RUN Parallel | 


RUN VAL 


Time for New 


1616 
% Std Dev New 
152 


C] Oracle? 


М Break When Not Matching 


idTransactionVersic idBox 


Improved by 41.34 
Improved by 35.05 


0 of 0 


Time for Old 
2488 

% Std Dev Old 
392 5 


Avg Off 11125 
2 Of 672 


BlockedCashAmow totalAmount 


0 0 usb 


153416125 53416125 (USD 


3800000 ¡USD 


(3800000 |usp 


| 53416125 153416125 USD 


w Projects |” Changes Agents 23 [] Build Queue 1 


INTEGRATED WITH CI/CD 


Bharat MANE |: 


ss FCC/TRA > ss Tomcat > ss БР Comparer > O Compile 8 Compare > © #282 (03 Nov 20 01:00) |< 


Overview 


Changes 2 


Build Log Parameters Dependencies 


VAL.ref_CommodityQuality_get 


[-] VAL.ref Counterpart get 


Timestamp 
11/3/2020 1:02:39 AM 
11/3/2020 1:02:39 AM 
11/3/2020 1:02:39 AM 
11/3/2020 1:02:39 AM 
11/3/2020 1:02:39 AM 
11/3/2020 1:02:39 AM 
11/3/2020 1:02:39 AM 
11/3/2020 1:02:39 AM 
11/3/2020 1:02:39 AM 
11/3/2020 1:02:39 AM 


11/3/2020 1:02:39 AM 


Result 
Data is not same for Table and for Column SG Rating 
Data is not same for Table and for Column SG Rating 
Row count is not same for Table 
Row count is not same for Table 
Data is not same for Table and for Column SG Rating 
Row count is not same for Table 
Data is not same for Table and for Column SG Rating 
Row count is not same for Table 
Row count is not same for Table 
Row count is not same for Table 


Row count is not same for Table 


Artifacts | Failure Performance 


New SP Time Old SP Time % Diff 
17 12 -5 
15 10 45 
16 13 -3 
19 14 -5 
21 24 3 
20 20 0 
27 47 20 
70 118 48 
45 35 -10 
16 12 -4 
31 58 27 


Command 
VAL.ref Counterpart get @idCounterpart = NULL, @filter ='2' , @activeOnly = NULL 
VAL.ref Counterpart get @idCounterpart = NULL, @filter ="Y' , @activeOnly = NULL 
VAL.ref Counterpart get @idCounterpart = NULL, @filter ='X' , @activeOnly = NULL 
VAL.ref_Counterpart_get @idCounterpart = NULL, (filter ="W' , @activeOnly = NULL 
VAL.ref Counterpart get @idCounterpart = NULL, (filter ="V' , @activeOnly = NULL 
VAL.ref Counterpart get @idCounterpart = NULL, (filter ='U' , @activeOnly = NULL 
VAL.ref Counterpart get @idCounterpart = NULL, (filter =T' , @activeOnly = NULL 
VAL.ref Counterpart get @idCounterpart = NULL, @filter ='S' , @activeOnly = NULL 
VAL.ref Counterpart get @idCounterpart = NULL, (filter = R , @activeOnly = NULL 
VAL.ref_Counterpart_get @idCounterpart = NULL, (filter ="Q' , @activeOnly = NULL 


VAL.ref_Counterpart_get @idCounterpart = NULL, (filter ='P' , @activeOnly = NULL 


REFACTOR 
FEARLESSLY / ` 


Gull 


REFACTOR 


11.626 SECONDS 01.08 SECONDS 


Statement Est Cost % | Compile Time | Duration + | CPU 
4 dbo.tmp testBundleFilter For old @valueDate = 2020-11-03 11:59:57, — date... SM, 11,626 аба ПИ а dbo.tmp_testBundleFilter_For_Val @valueDate = '2020-11-03 11:59:57, — date... 100.0% 1,807 188 100.0% 
4 EXEC dbo.[prv. BundleFilter set] @valueDate = @valueDate, -- datetime Da... 99,2% 10,630 468 4 | EXEC val.[prv BundleFilter set] &valueDate = GQvalueDate, — datetime Did... 99,1% 1,476 172 98.9% 
4 EXECUTE sp_executesgl @QueryStatementBundleNullBlock, &parameter... 7.5% 1,417 46 INSERT INTO bundleLocal (idBundle, idBundleUp, iditemType, [type], v... 30.7% 47 76 74 54.6% 
7.5% 62 68 29 UPDATE &CombinedFacility SET isActive = 1 WHERE idBBMultiFacilityHea. .. 0.6% 36 39 29 0.1% 
» | INSERT INTO @foCodesTableParam SELECT * FROM fn GetDistinctFOCo... 0.1% 0 185 2 INSERT INTO #bundleLocal ( idBundle, idBundleUp, idltemType, [type], v... 21.7% 146 15 13 19.9% 
INSERT INTO #bundle 1 ( idBundle, idBundleUp, Ift, rat, idTree, valueDat... 15.9% 56 166 66 INSERT INTO #bundleLocal ( idBundle, idBundleUp, idItemType, [type], v... 13.8% 32 10 9 10.4% 
DELETE FROM #bundle 1 WHERE idBundle IN (SELECT idBundle FROM @... 0.8% 5 134 0 INSERT INTO #bundleLocal (idBundle, idBundleUp, idItemType, [type], v... 21.9% 49 10 10 11.6% 
UPDATE =bundle1 SET type = 'Fadlity', idltemType = 2, - Ref ItemType... 4.4% 33 85 28 INSERT INTO @CombinedFacility SELECT DISTINCT m.idBundle, m.idBBM... 1.0% 4 7 1.2% 
UPDATE B SET type = ‘Box’, idltemType = 3, —Ref ItemType idKey = C... 13.2% 25 77 39 INSERT INTO bundle (idBundle, idBundleUp, idltemType, [type], value... 8.5% 0 3 2 0.1% 
UPDATE 8CombinedFadiity SET isActive = 1 WHERE idBBMuliFadiityHea... 0.2% 31 60 28 INSERT INTO @foCodesTable SELECT + FROM fn_GetDistinctFOCodes(... 0.1% 0 1 1 0.0% 
UPDATE В SET minNCR = CASE WHEN B.idBundle = F.idBundle AND FC.id... 13.9% 9 39 39 DROP TABLE #authorizedFadlities; Aer, 5 0 мү 
UPDATE В SET hasMarginCaliThreshold = FC.hasMarginCallThreshold, thr... 4.5% 6 23 17 DECLARE @CUSTOMER_ITEM_TYPE INT = 1; 0.0% 0 0 0.0% 
INSERT #bundle SELECT * FROM #bundle1 7.9% 3 18 9 DECLARE @FACILITY ITEM TYPE INT = 2; SE - à SE 
UPDATE #bundle1 SET type = ‘Customer’, idItemType = 1, 一 Ref ItemT... 8.3% 10 12 11 m €—Á—MÓÁá— 3 225 = m SE 
UPDATE B SET localBL = БЇБЇ label FROM #bundle1 B INNER JOIN Ref L... 4.4% 4 12 9 ee ROME aa св: "p = е Tm 
SELECT @IsTom9116Enabled = [Enabled] FROM dbo.FeatureToggle WHE... 0.0% 0 9 0 | = — е. 

UPDATE B SET FrontOfficeCode = RFOC.label FROM #bundle1 B INNER 1... 4.3% 4 9 7 вагон по реге а EEN e Е en 
INSERT INTO #bundle1 ) idBundle, idBundleUp, Ift, rat, idTree, valueDat... 4,2% 14 8 8 A AREA isc o = 
INSERT INTO #bundle1 ( idBundle, idBundleUp, Ift, rat, idTree, valueDat... 4.1% 10 7 6 DECLARE @FACILITY_STATE_INACTIVE INT = 0; 0.0% 0 0.0% 
UPDATE #bundle 1 SET type = 'CombinedFacility’, iditemType = 17, — Re... 1.5% 14 4 4 DECLARE iisActive INT = 1; 0.0% 0 0.0% 
UPDATE #bundle 1 SET ca3isodevEOD = ISMULL(CF.Currency, C.ca3isod... 1.3% 8 3 3 DECLARE @MaxDate DATETIME = "9999-01-01; 0.0% 0 0.0% 
INSERT INTO @CombinedFacility SELECT DISTINCT m.idBundle, m.idBBM... 0.8% 4 2 2 CREATE TABLE #bundleLocal ( idBundle INT, idBundleUp INT, idItemType ... 0.0% Ü 0 0.0% 
k INSERT INTO @foCodesTable SELECT * FROM fn. GetDistinctFOCodes(... 0.1% 0 1 1 CREATE TABLE #authorizedFadlites ( [idFacility] INT NOT NULL, [idBundl... 0,0% 0 0 0.0% 
k SELECT = INTO #bundle1 FROM dbo.ts Bundle create() 0.1% 1 1 1 IF (@idBundle 15 NOT NULL) 0.0% Ü Ü 0.0% 
UPDATE #bundle1 SET ca3isodevEOD = ISNULL(ca3isodevEOD, ca3isode... 0.7% 3 1 1 IF OBJECT. ID( tempdb.. #authorizedFadlities”) 15 NOT NULL 0,0% 0 0.0% 
IF (@idBundle IS NOT NULL) 0,0% 0 0 SELECT idBundle, idBundleUp, idItemType, valueDate, ІН, rot, idTree FROM ... 0.8% 211 2 1.1% 
IF (@IsTom9116Enabled = 1) 0.0% 0 0 4 SELECT * INTO #bundle FROM val.ts Bundle create temp table() 0,1% 2 0 0.0% 
IF (@isActive 15 NULL) 0.0% ü 0 RETURN 0.0% 0 ü 0.0% 
SET fisActiveCondition = № (( @isActiveParam = 1 AND (F.idFacilityStat... 0.0% 0 0 DECLARE @isActive INT = 1; 0.0% 0 0.0% 

IF (@localizationSG 15 NULL) 0.0% 0 0 

SET @localizationSGCondition = N' '; 0.0% Ü ü 

IF (@idFOCode IS NULL) 0.0% 0 0 

SET @idFOCodeCondition = N' '; 0.0% 0 0 

IF (@isMigrated IS NULL) 0.0% 0 0 


WHILE ADDING 


NEW CODE 


IS MUST 


WHEN YOUR CODE WORKS THE FIRST TIME 


IT S JUST HALF THE JOB DONE 


THE TIME THAT THE CODE FIRST 
WORKS... 


«IT'S THE MOMENT IN WHICH 
YOU START BUILDING YOUR 
CAREER 


IT’S THE MOMENT IN WHICH 
YOU START BUILDING YOUR 
DESIGN SKILLS 


MICRO SELF 
DOUBTS REMORSE DECISIONS REVIEW 


CG) 22 
o 


s OM 


IT'S PERFECT MOMENT TO 
CLEAN THE CODE 


NAMES FUNCTIONS COMMENTS 


OBJECTS 


FORMATTING я ERROR CLEAN CODE 


DATA HANDLING 


STRUCTURES TOPICS 


CONTINUOUS PAIR 


REFACTORING PROGRAMMING шайы, 


NAMES 


THE POWER INVESTED IN OUR 
FINGERTIPS 


local variable private function 


instance variable . 
argument 5 Cl a S S 
namespace glo pal variable public function 
module library 


„BECAUSE WE NAME SO 
MUCH 


„МЕ MUST USE THIS 
POWER WISELY 
WE MUST DO IT WELL 


COINED THE PHRASE, 
“IMPLICITY OF A CODE” 


TIM OTTINGER 


USE INTENTION-REVEALING NAMES 


int d; // elapsed time in days — int elapsedTimeInDays; 

public List«int[]» getThem() 4 public List<int[]> getFlaggedCells() { 

List<int[]> listi = new ArrayList<int[ ]>(); List<int[]> flaggedCells = new ArrayList«int[]»(); 

for (int[] x : theList) for (int[] cell : gameBoard) 

if (x[@] == 4) — if (cell[STATUS VALUE] == FLAGGED) 
listl.add(x); flaggedCells.add(cell); 

return 11511; return flaggedCells; 

} } 


public List<Cell> getFlaggedCells() í 
List<Cell> flaggedCells = new ArrayList<Cell>(); 
for (Cell cell : gameBoard) 
if (cell.isFlagged()) 
flaggedCells.add(cell); 
return flaggedCells; 


WE COMMUNICATE 


LENGTH 


Se dete le le be № ПО RAR A Р ШЫП 
0 11 12 13 14 12 13114 115 116 117 118 119 120 121 122 123 124 
mm 


VARIABLE NAME 
LENGTH 


ITS SCOPE 


FUNCTION NAME 
LENGTH 


ITS SCOPE 


CLASS NAME 
LENGTH 


ITS SCOPE 


V 1.0 V 2.0 V 3.0 V 4.0 


ä Tomcat. StoredProcedure 4 Tomcat.SPComparer b “|| Tomcat.SPComparer 
b ж Pro pe = р А Properties 4 了 [Et To A rerProwider 
b им References b 5, Properties 
P ЕШ References mE b шш References 
| А с. Config 
v1 App.config b SPs b все CommandOutput.cs 
_ P + С ComparerContiguration.cs 
b EZ] Forml.cs LT serverconfig.json wit desse dein 
p С“ Program.cs لو‎ App.config b a C* Comparingltem.cs 
р CH Uti lity CS b се Data5setComparer.cs P ас ComparinglternCollection.cs 
ро се DBltem.cs b ас“ DataSetComparer.cs ? ? ? 
b се Logger.cs b ač DataSetComparerResult.cs 
+] packages.config p ac DEREN 
dido as (тб Emcuionlimecs 
> Ez] SPComparer.cs b +C* [StoredProcedureExecutor.cs 
P се SPComparerOutput.cs b š ca Logger.cs 
р E StoredProcedureComparer.cs b ac SPComparerOutput.cs 
P се Utility.cs b a C* StoredProcedureComparer.cs 
b +C* StoredProcedureExecutor.cs 
P +C* StoredProcedureQutputComparer.cs 
b ec Utility.cs 


Р Tomcat.StoredProcedurelester 


P 
р 


A 


чт о т w x Së s 


А Properties 

SR References 

| Interfaces 

P Ce [Executor.cs 

р се [TestResultPublisher.cs 

m! Model 

CS DataSetComparerResult.cs 

се MissMatchltemType.cs 

се StoredProcedurelnExecution.cs 
се TestConfig.cs 

се TestUutput.cs 

се TestParameter.cs 

се TestSurite.cs 

се TestSuiteQutput.cs 

се TestUínit.cs 

се [estUnitOutput.cs 

се TestUnitRundutput.cs 
+) App.config 

се DataSetComparer.cs 

CH HtmlTestResultPublisher e 
C* Loader.cs 

се MSSOLExecutor.cs 
се Tester ce 


C* TestRunner.cs 


что ST wx чт Së т у x sx s 


THE MOMENT NAME 

WAS GIVEN ТО THIS 

CLASS THE WHOLE 
THING BECAME 
CRYSTAL CLEAR 


GOOD NAMES TELLS 
THE CONTEXT OF 
ENTIRE SYSTEM 


WE COMMUNICATE 


AVOID DISINFORMATION 


public class Document 
1 lab] public class DocumentInfo 
Маг WFÉFOOPGES EOSSCOPEXHDAd public class DocumentDetail 


var XYZFooBarClassForBlablable public class Collateral 


public class Documents 
public class DocumentList 


var a = 1; Using lower case | 
if ( O = 1) (looks like number-1) and 
| de Uppercase O (looks like 


number-0) are also 
unhelpful. 


“A software author must avoid leaving false clues 
which obscure the meaning of code.” 
- Ottinger 


WE COMMUNICATE 


USE PRONOUNCEABLE NAMES 


// generation date, year, months, day, hour, minute, and second // better: 

class DtaRcrd182 4 class Customer { 
private Date genymdhms; — private Date generationTimestamp; 
/ * шп m » / / * u m m ж i d 

} } 


— CURRENCY 


aval ueDate EM 
@expiryDate datetime , 


public void CalculateSCPUAvailableQuantity(DocumentTransfer transfer) 


“Programming is a social activity” - Bob Martin 


WE COMMUNICATE 


USE SEARCHABLE NAMES 


int realDaysPerIdealDay = 4; 
const int WORK DAYS PER WEEK = 5; 
int sum = 0; 


tor (int j=8; 1434; j++) + for (int j=0; j < NUMBER OF TASKS; j++) { 


a 2 — و‎ 5 " Е. 
5 += (t [3 ] *4)/ fe int realTaskDays = taskEstimate[j] * realDaysPerIdealDay; 
1 int realTaskWeeks = (realdays / WORK DAYS PER WEEK); 
sum += realTaskWeeks; 
} 
const NO SEED = @ 
const ONE_SEED = 1; 
let a = {'арр1е' : 2, 'mango' : 1, const MANY_SEEDS = 2; 
‘banana’: 8, ‘orange’: 2, 
'water-melon': 2} let fruits = ('apple' : MANY SEEDS, ‘mango’ : ONE SEED, 
banana NO SEED, ‘orange’: MANY SEEDS, 
for (b in a) ( 'water-melon': MANY SEEDS) 
if (a[b] == 2) с = 'many' // print the seeds count 
else if (a[b] == 1) c = ‘one’ for (fruit in fruits) 1 
else if (a[b] == 8) c = 'no' let count = 'no'; 
console.log(a, 'has', c, c == 'many'?  'seeds' : 'seed') if (fruits[fruit] -- MANY SEEDS) count - 'many' 
} else if (fruits[fruit] == ONE SEED) count = 'one' 
console.log(fruit, 'has', count, count == 'many'? 'seeds' : 'seed') 


Replace literals with constants 


CLASS NAMES 


A CLASS NAME SHOULD 
BE A NOUN, 
NOT A VERB. 


AVOID WORDS like 
Manager, Processor, Data, or Info. 


GOOD NAMES could be: 
Tester, TestSuite, TestUnit, HtmlTestResultPublisher. 


METHOD NAMES 


METHODS SHOULD HAVE VERB 


internal static void Consolidation(Process process, 
internal static void UnitDdaNetting( 
internal static void UnitAnalyticalOutstandingGeneration(Process process, 


public double QualityUnitConversion(int quality, int idUnit, double quantity, 
int sourceDocumentIdQuality, 
int sourceDocumentIdUnit, 
DateTime valueDate)|..| 


GOOD NAMES could be: 
Save(), Run(), Publish(), Valuate(), 
Consolidate() 
RunNettingValuation(), 
GenerateAnalyticalOutstanding() 
ConvertQualityUnit() 


WE COMMUNICATE 


HOWEVER 


THERE ARE NO GOOD NAMES 


IT'S CONTINEIOUS PROCESS 
EVERY TIME YOU HAVE BETTER 
UNDERSTANDING OF CODE, 
YOU FIND A BETTER NAME 


ENGLAND'S ETON COLLEGE 


"THAT'S EASY" 


JUST BRUSH MOW THEM AND ROLL 
OFF THE DEW EVERY THEM 
EVERY MORNING OTHER DAY ONCE A WEEK 
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NEGLECTION ACCELERATES ROT 
FASTER THAN ANY OTHER FACTOR 


TEST AUTOMATION GIVES YOU 
POWER OF 
FEARLESS REFACTORING 


FIND OPPORTUNITIES OF 
AUTOMATION IN ABSENCE OF 
TESTS FOR OLD CODE 


USE THE GIFTED POWER OF 
NAMING THINGS WISELY 


USE INTENTION-REVEALING NAMES 


AVOID DISINFORMATION 


USE PRONOUNCEABLE NAMES 


USE SEARCHABLE NAMES 


CLASS NAMES MUST BE NOUNS 
METHOD NAMES MUST BE VERBS 


REMEMBER WE DON'T CODE, 
WE COMMUNICATE 


REMEMBER WE DON'T CODE, 
WE COMMUNICATE 


DON’T FEAR 


& 
TAKE CARE 


MINDSET 


WANT TO DO IT 


ANNEX | URE 


GOOD READS 


Robert С. Martin Series ' : 
қ Robert С. Martin Series 


Clean Code Craftsmanship The Clean Coder 


A Handbook of Agile Software Craftsmanship | - A Code of Conduct for Professional Programmers 


7 The New 
°C) Imperative 


Martin Fowler 


wih contributions by your journey (9 mastery 
Kent Beck | |! | 
| 


| 


\ 
| 
i 


Pete McBreen 
Foreword by Dave Tbomas SECOND EDITION 
Robert C. Martin 


DAVID THOMAS 


ANDREW HUNT 
Robert C Martin 


Design Patterns 


Robert C. Martin Series 


Fleme ot Reusable 


TI EST- DRIVEN + ` | ( лесі: t-Oriented Software 
DEVELOPMENT | Helm 


The Software 
` Craftsman 


| Professionalism, Pragmatism, Pride 
1 


hard Helm 
pi He NI 
n Visados 


WORKING E "т. 
EFFECTIVELY | 
WITH 


LEGACY CODE 


Michael C. Feathers 


[wmm 


Sandro Mancuso 


principles Grind that 


patterns knowledge 
practices Into muscle 
heuristics memory 


KNOWLEDGE WORK CRAFTSMANSHIP 


“HONESTY IN SMALL THINGS 
IS NOT A SMALL THING.” 


-Danish Saw 


Refactoring 


When refactoring every change you make 
is a small behavior-preserving change. You 
only refactor with green tests, and any test 
failing indicates a mistake. By stringing 
together a series of small changes like this 
you can move more quickly and with less 
risk because you shouldn't get trapped in 
debugging. 


Adding Function 


Any other change to the code is adding 
function. You will add new tests and break 
existing tests. You aren't confined to 
behavior-preserving changes (but it's wise 
to keep changes small and return to green 
tests swiftly). 


During programming you may swap 
frequently between hats, perhaps every 


couple of minutes. But... 


You can only wear one hat at a time 


UNIT OF “' Gë 
CODE QUALITY (|) | Кк. тер, 


(> Ө | ww ( | 


(с) 2008 Focus Shift/OSNews/ Thom Holwerda - http: / /www.osnews.com /comics 


Maintenance == 


Design Done 


Cost of a Change 
Wall of Unmaintainability 


Time 


When we proclaim the design is done and accept no more 
changes 


When we move the system into maintenance and change the 
team's process 


THERE ARE ONLY TWO HARD THINGS IN 
COMPUTER SCIENCE: 
CACHE INVALIDATION AND NAMING THINGS 


MOST OF THE IDES GIVES YOU VERY 
SAFE WAY TO RENAME 
VARIABLES, CLASSES, FUNCTIONS 
ESPECIALLY PRIVATE STUFF IS 
COMPLETELY SAGE 


Logger | logger; 
string © +| parameters; 


int _F Rename ' currentLogger to '_logger 四 zd 
e  InitRun([); 
Lon Jõürremtlogger = new Logger(Utility.GetAssemblyDirectory() + "А" + cm 
= new Logger(fUtility.GetassemblyDirectory() + 75" + cmbśps.5e. 
await Tassk.Run(() => Compare()}; 


ctors m 
btnParallel.Enabled = false; 

icurrentLogger = new Logger(Utility.GetAssemblyDirectory() + @"\" + cm 
new Logger(Utility.GetássemblyDirectory[) + BX" + cmbSps.Se. 


Event Handelers 


ger.Info(” " + logMessage + output.ExecutionTime.TimeTal 
^ + logMessage + output .ExecutionTime. TimeTakendylor 
output. ExecutionTime. TimeTakenbytomparekith. To! 


void SPComparer_Load(object ser 


COMPREHENSION 


Re public Cage(double Length, double width, double D 1 


jth = width; 
igth = Length; 


ght = height; 


public double getVolume(){ 
return width * Length * height; 
} 


к public static void main(String[] args) 4 


Rename usages of 'height' to 'h' 


Cage cage = new Cage( length: 1.0, width: 2.0, 
System.out.println(cage.getVolume()); 


REFACTORING 


h: 5.0); 


MANIFESTO 
FOR SOFTWARE 
CRAFTSMANSHIP 


EXTREME PROGRAMMING PRACTICES 


Group 


Feedback 


Continual Process 


Code understanding 


Work conditions 


Practices 
v Test-Driven Development 
v  ThePlanning Game 
У On-site Customer 
Y 


> > + 


> >> + 


Pair Programming 


Continuous Integration 
Code Refactoring 
Small Releases 


Simple Design 

Collective Code Ownership 
System Metaphor 

Coding Standards 


40-Hour Week 


XP has simple rules that are based on 5 values. 


О) COMMUNICATION 


Ф 


Q) 
Q) 
Q) 


Everyone on a team works jointly at every stage of the project 


SIMPLICITY 


Developers strive to write simple code bringing more value to 


a product, as it saves time and efforts. 


FEEDBACK 


Team members deliver software frequently, get feedback about 
it, and improve a product according to the new requirements. 


RESPECT 


Every person assigned to a project contributes to a common 


goal. 


COURAGE 
Programmers objectively evaluate their own results without 


making excuses and are always ready to respond to changes. 


